stylecontext: Do not transition from the default style

This gets rid of a bunch of awkward transitions. It's not a perfect
solution to the problem of "should we transition from this state" but it
gets rid of the ugliest offenders.

Adwaita: scrollbar style refactoring...

...making overlay indicator slider visible on any backgroung
in the process.

Updated Czech translation

widget: Free the controller list on finalize()

If this is done on dispose(), the widget may be destroyed (and its
controllers list NULLified) within _gtk_widget_run_controllers(),
causing warnings/crashes when it just tried to hop on the next
controllers.

Freeing the controllers here should be a safety net for implementations,
so it also makes sense to do this late. The widgets that choose to
free their controllers on dispose can still do so, and get
_gtk_widget_remove_controller() called for these as an indirect result.

listbox: Free the multipress gesture on finalize

Updated Icelandic translation

event-controller: Chain up on dispose

Updated Russian translation

gtkplacessidebar: insert bookmark by dnd in correct position

Add needed COLUMN INDEX to "new bookmark" row based on previous
bookmark index plus one.

Fixes bug 742727

Fix accessible_class->widget_set initialization

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742778

Updated Galician translations

Updated Spanish translation

Properly check result of g_utf8_get_char_validated()

g_utf8_get_char_validated() may return -1 or -2
return type is gunichar(guint32)

Therefore such checks like 'gunichar < 0' or 'gunichar > 0'
are always 'false' or 'true'(except when gunichar == 0).

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742774

Fix check for 'y' padding in gtk_cell_renderer_set_padding()

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742764

Remove an unused define

Properly ignore BadWindow and BadDrawable in get_child_info_handler()

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742771

Forgotten file

HighContrast: Don't set nonexisting style properties

These don't exist, so setting them can't do any good.

Adwaita: Don't set nonexisting style properties

These don't exist, so setting them can't do any good.

HighContrast: Also fix separators in file chooser buttons

Same fix as for combo box separators.

Adwaita: Also fix separators in file chooser buttons

Same fix as for combo box separators.

gtkfilechooserwidget: When using an extra widget, show or hide the filter combo as appropriate

https://bugzilla.gnome.org/show_bug.cgi?id=742653

HighContrast: draw combo box separators

Same fix as the previous commit.

Adwaita: draw combo box separators

Killing the separator in the button had the unintended side effect
of also killing separators in the popup.

https://bugzilla.gnome.org/show_bug.cgi?id=742704

widget-factory: Add a combo box separator example

This shows that combo box separators are currently broken
in Adwaita.

GtkTextView: allow to extend text selection with mouse scrollwheel

Fixes bug 710612

file chooser: Add some padidng

https://bugzilla.gnome.org/show_bug.cgi?id=742631

Forgotten files

Adwaita: Fix treeview separators in backdrop

https://bugzilla.gnome.org/show_bug.cgi?id=742702

stylecontext: Stop animations on manual invalidation

If a gtk_style_context_invalidate() is called on a widget's style
context (which nobody should ever do, sheesh!) and we're animating, stop
the animations.

Fixes crashers in Nautilus.

GtkScrolledWindow: Don't start fading right away

When we get a leave event, we need to remove the over bit,
since we may not get motion events that tell us when we lose
proximity, but we don't have to fade out right away - that
will happen on its on due to inactivity a bit later. Having
the indicator stay visible for a bit feels more relaxed.

GtkScrolledWindow: Factor out a constant

Define INDICATOR_CLOSE_DISTANCE instead of hardcoding the same
number of pixels in 4 different places.

GtkScrolledWindow: Don't require proximity for track points

These are hard to navigate precisely, so show wide scrollbars
right away.

GtkScrolledWindow: Drop device manager complications

We no longer use the presence or absence of particular device
types to determine which scrolling to use.

GtkScrolledWindow: Drop a redundant field

The Indicator.enabled field was not carrying any extra information,
so just use priv->use_indicators instead.

GtkScrolledWindow: Rename a private field

touch_mode -> use_indicators, more to the point.

x11/gl: Do not specify the GLX context version

If we use GDK_GL_PROFILE_3_2_CORE we are asking for a core profile
according to the GLX_ARB_create_context_profile extension. For that,
we pass the GLX_CONTEXT_CORE_PROFILE_BIT_ARB value for the
GLX_CONTEXT_PROFILE_MASK_ARB attribute.

The specification for the extension says that:

  If the requested OpenGL version is less than 3.2,
  GLX_CONTEXT_PROFILE_MASK_ARB is ignored and the functionality
  of the context is determined solely by the requested version.

Since we're asking for a core profile, we assume a GL version greater
than or equal to 3.2; thus, we don't need to specify the
GLX_CONTEXT_MAJOR_VERSION_ARB or the GLX_CONTEXT_MINOR_VERSION_ARB
attributes, and instead just rely on whatever version GLX gives us.

This seems to work around a strange issue in Mesa; if we ask for a core
profile and any version > 3.0, we get broken rendering on any shared
context we create.

inspector: Don't use GSettings directly

It is not necessary here, and using GtkSettings gives us
a greater chance to not fail e.g. on Windows.

https://bugzilla.gnome.org/show_bug.cgi?id=742664

inspector: be careful about gsize vs guint64

Since gsize doesn't work as a typename in a ui file,
we have to be careful not to pass pointers to wrongly
sized variables when getting the guint64 values out
of the model.

https://bugzilla.gnome.org/show_bug.cgi?id=742664

docs: Add missing quote

The missing " breaks the syntax highlighting of the GtkImageMenuItem
example.

docs: Add Since for GtkCellRendererPixbuf:surface

docs: Fix some old live.gnome.org URLs

Updated POTFILES.skip

Fix GTK_OVERLAY_SCROLLING=0

This was broken by the recent more dynamic approach.

Fix comment typo

inspector: Show object address

This is useful information when running the application
in a debugger.
https://bugzilla.gnome.org/show_bug.cgi?id=742518

Point to copy/cut/delete named icons in the stock item deprecations

We still use symbolic icons for Copy, Cut, and Delete operations inside
toolbars and some menus, so it's worth pointing to the symbolic icon
names in the deprecation notices.

https://bugzilla.gnome.org/show_bug.cgi?id=741969

file chooser: Don't force a scrollbar

No point in showing the scrollbar when it is not needed.
https://bugzilla.gnome.org/show_bug.cgi?id=742343

gtklabel: Don't accept unhandled mouse buttons

If the pressed mouse button neither triggers a context menu nor
activates a link, set the sequence's state to DENIED.

https://bugzilla.gnome.org/show_bug.cgi?id=742010

cssimage: Fix angle computation for gradients to corners

Updated Basque language

Updated Basque language

stylecontext: Handle theme changes for cached styles

We need to clear the cache manually on full revalidates because
_gtk_css_change_for_child() will clear the full revalidation flags.
And then gtk_style_context_update_cache() will not do the right thing
(which is to clear itself).

stylecontext: Rename GtkStyleInfo to GtkCssNode

stylecontext: update_animating() properly

Because we can switch from animating to non-animating pretty much
anywhere, do the check for animations unconditionally instead of trying
to cram it into the correct if path (and failing).

stack: Remove commented out line

It always shows up in my greps and confuses me.

stylecontext: Add a parent style cache

We now cache the results of lookups on the parent GtkCssStyle. This
allows sharing styles between widgets (recursively). However, this
only works if the styles can't potentially depend on siblings -
neither directly via sibling selectors or via position pseudo-classes
like :first-child.
Unfortunately, Adwaita currently uses first-child a lot, and in
particular for labels, which are the most common widgets.

The big benefits of this change are both less CPU - due to not needing
to compute styles again - and less memory usage - due to sharing of
the styles between widgets. Here's some nonscientific numbers I
collected while pondering the usefulness of this patch:

                    glade   glade   widget
                    demo    demo    factory
                    runtime styles  styles
Adwaita before      19.1s   5800    1150
Adwaita now         18.9s   3800     970
Adwaita hacked now  14.5s   3100     910
simple before       11.3s   5800    1150
simple now          10.8s   1300     590

Adwaita: Adwaita as provided by GTK
Adwaita hacked: Adwaita with the first/last-child for GtkLabel removed
simple: A 250 lines simple GTK theme I use for testing
before: This patch not applied
now: this patch applied

glade demo runtime: Starting glade opening a large file and closing it
glade demo styles: GtkCssStaticStyle objects after opening glade with
                   the large file as per inspector
widget factory styles: GtkCssStaticStyle objects after startup as per
                       inspector

cssstyle: Have a default style

This way we can initialize the default style info with a style and
ddon't need to special case NULL everywhere.

stylecontext: Use correct parent on updates

Previously we looked at the save/restore state when determining the
parent. This is wrong in the case where we're updating the cache though.

So we now save the parent in the style info.

stylecontext: Fix cache updating

After the parent changes in commit
3a337156d11a86c7a88f1f30a09276fdf6f63008 we need to refresh the cached
styles after the current style. After all, they now depend on the base
style.

cssstaticstyle: Handle relevant change

Relevant change is returned from style lookups, so keep is with the
object that holds those values.

stylecontext: Use correct parent style for lookups

After 3a337156d11a86c7a88f1f30a09276fdf6f63008 style lookups still used
the parent context's style as the parent style, even though after a
gtk_style_context_save() the root style of the style context is the
proper parent.

Testcase attached.

cssnodedeclaration: Store the GType of the node

cssstyle: Move function from vfunc to static func

compute_dependencies() is only used internally by GtkCssStaticStyle,
so there's no need to have it elsewhere.

cssstaticstyle: Move function

And with that move, GtkCssStaticStyle is immutable.

cssstaticstyle: Move function

We want the new() return values to be immutable, so we have to move the
code that modifes them.

stylecontext: Refactor update_properties()

It now always returns a new instance.

stylecontext: Simplify function

Instead of computing the changes manually everywhere from the parent
changes, just pass in the parent changes.

stylecontext: Simplify code

Compute the differences only once and not in both if branches.

cssanimatedstyle: Apply animation values after creation

This makes GtkCssAnimatedStyle immutable.

cssanimatedstyle: Make advancing the style return a new style

This is another step towards making CssStyles immutable.

stylecontext: Fold a function into only caller

cssanimatedstyle: Avoid creating animation

... when no animation exists.

This makes the function name kinda wrong, but I'm not sure what else to
call it.

cssanimatedstyle: Remove unused function

cssanimatedstyle: Merge two functions

stylecontext: Make build_properties create the style

stylecontext: Make static styles static

Instead of keeping an animated style everywhere, we replace it with the
static style when nothing gets animated.

Apart from making the code cleaner, this gets rid of a bunch of animated
style values that do nothing but wrap a static style.

stylecontext: Split function

stylecontext: Only look up stuff in the cache that can be in the cache

Only saved styles are stored in the cache so we should look only those
up there.

stylecontext: don't unset the current style info when clearing cache

Instead, do it manually.

stylecontext: Always pass the static style

... to build_properties.
There is always one available, so we can insist on that one.

This simplifies a bunch of code.

stylecontext: Use a static style when saved

The only style that is animated is the style of the unsaved primary
node. So there's no need to create animated style objects for the other
ones.

There is a bunch of ugliness in the code currently. Further commits are
expected to fix them.

cssstyle: Add GtkCssStaticStyle

For now, this is only an implementation detail of the animated style.

The idea is to use GtkCssStaticStyle as the result of CSS queries and
then put a GtkCssAnimatedStyle on top that manages the animations. The
neat thing about this is that you can cache the static values.

cssstyle: Split into GtkCssStyle and GtkCssAnimatedStyle

GtkCssStyle is the base class to be used for all types of styles that do
exist.

GtkCssAnimatedStyle is the only implementation so far, that is exactly a
copy/paste of the old GtkCssStyle code.

cssstyle: Rename GtkCssComputedValues => GtkCssStyle

This is literally just renaming of the object (and the associated source
files). No other changes are in there.

cssstylefuncs: Rename functions

This is in preaparation for the big GtkCssComputedValues => GtkCssStyle
rename

Updated German translation

Updated British English translation

cssselector: Reintroduce GOT_MATCH

I removed it in 14f5ce710856708dc80fcc98cb5eb2ba257f34ad because I
thought it was unnecessary, but it wasn't. When we build a tree like
this:

.matches ─┬─ .doesntmatch
          └─ .alsodoesntmatch

We would get the changes for the .matches part returned. This is however
only right if that node of the tree contains results. If results only
exist with the child nodes (all of which don't match), then this part
should not match either.

cssselector: The if was the wrong way around

We want to keep matching as long as selectors are simple. Otherwise we'd
never be matching, because the first selector is always simple. Oops.

cellrenderertext: notify inverted

The notify for "size" and "size-points" properties
are inverted.

https://bugzilla.gnome.org/show_bug.cgi?id=742311

calendar: Handle invalid dates

GtkCalendar can have an invalid date — mostly at initialization. This
means that GDateTime construction may fail. We need to handle that case
gracefully, like the old code did.

This fixes the `notify` test suite, which started failing with:

/Notification/GtkCalendar:
GLib-CRITICAL **: g_date_time_get_day_of_week: assertion 'datetime != NULL' failed

inside the Continuous builder.

textview: fix duplicated binding entries. use kp variants instead.

This appears to have accidentally used the normal keypresses twice instead
of the KP variants. This resulted in shift+insert pasting twice.

https://bugzilla.gnome.org/show_bug.cgi?id=742170

Updated Basque language

Updated Hungarian translation

Updated Hungarian translation

Updated Hungarian translation

Updated Vietnamese translation

Signed-off-by: Trần Ngọc Quân <vnwildman@gmail.com>

Fix typos in documentation

gtkstylecontext: Don't try to emit a signal when finalizing